Skip to content

fix: migrate an empty legacy pending-delivery table without data loss - #384

Merged
abrichr merged 2 commits into
mainfrom
claude/tv2-legacy-pending-migration
Aug 21, 2026
Merged

fix: migrate an empty legacy pending-delivery table without data loss#384
abrichr merged 2 commits into
mainfrom
claude/tv2-legacy-pending-migration

Conversation

@abrichr

@abrichr abrichr commented Aug 21, 2026

Copy link
Copy Markdown
Member

Summary

  • a run directory written before the terminal-verification v2 durable authority carries a remote_delivery_pending table without the authority_origin column
  • an empty legacy table now gains the column in place, so an old run directory still validates
  • a populated legacy table still refuses with DurableAuthorityBusy: no schema migration ever rewrites a live pending delivery

Verification

  • tests/test_durable_authority_v13.py: 58 passed (two new tests cover the empty migration and the populated refusal)
  • tests/test_terminal_verification_v2.py: 36 passed
  • ruff clean on both touched files

A run directory written by the pre-v2 durable authority carries a
remote_delivery_pending table without the authority_origin column.
An empty legacy table now gains the column in place; a populated one
still refuses with DurableAuthorityBusy so no live pending delivery
is ever rewritten by a schema migration.
@abrichr
abrichr merged commit d73e629 into main Aug 21, 2026
16 checks passed
@abrichr
abrichr deleted the claude/tv2-legacy-pending-migration branch August 21, 2026 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant